+Mon Aug 25 23:21:43 2003 Kristian Rietveld <kris@gtk.org>
+
+ Merge from stable.
+
+ Fixes #115871, reported by Michael Natterer.
+
+ * gtk/gtktreeprivate.h: added GtkTreeSelectMode enum,
+ added ctrl_pressed and shift_pressed bitfields,
+ (_gtk_tree_selection_internal_select_node): replace GdkModifierType
+ arg with GtkTreeSelectMode.
+
+ * gtk/gtktreeselection.c (gtk_tree_selection_set_mode),
+ (gtk_tree_selection_select_path), (gtk_tree_selection_unselect_path),
+ (_gtk_tree_selection_internal_select_node): all updated for
+ GdkModifierType -> GtkTreeSelectMode move.
+
+ * gtk/gtktreeview.c (gtk_tree_view_button_press): set ctrl_pressed
+ and shift_pressed around selection handling block,
+ (gtk_tree_view_real_select_cursor_row),
+ (gtk_tree_view_real_toggle_cursor_row),
+ (gtk_tree_view_real_selection_cursor_parent),
+ (gtk_tree_view_real_set_cursor): use ctrl_pressed and shift_pressed,
+ instead of checking the event state. And also updates for the
+ GdkModifierType -> GtkTreeSelectMode move.
+
Wed Aug 20 21:26:49 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkplug.c (gtk_plug_focus): Send the focus on
+Mon Aug 25 23:21:43 2003 Kristian Rietveld <kris@gtk.org>
+
+ Merge from stable.
+
+ Fixes #115871, reported by Michael Natterer.
+
+ * gtk/gtktreeprivate.h: added GtkTreeSelectMode enum,
+ added ctrl_pressed and shift_pressed bitfields,
+ (_gtk_tree_selection_internal_select_node): replace GdkModifierType
+ arg with GtkTreeSelectMode.
+
+ * gtk/gtktreeselection.c (gtk_tree_selection_set_mode),
+ (gtk_tree_selection_select_path), (gtk_tree_selection_unselect_path),
+ (_gtk_tree_selection_internal_select_node): all updated for
+ GdkModifierType -> GtkTreeSelectMode move.
+
+ * gtk/gtktreeview.c (gtk_tree_view_button_press): set ctrl_pressed
+ and shift_pressed around selection handling block,
+ (gtk_tree_view_real_select_cursor_row),
+ (gtk_tree_view_real_toggle_cursor_row),
+ (gtk_tree_view_real_selection_cursor_parent),
+ (gtk_tree_view_real_set_cursor): use ctrl_pressed and shift_pressed,
+ instead of checking the event state. And also updates for the
+ GdkModifierType -> GtkTreeSelectMode move.
+
Wed Aug 20 21:26:49 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkplug.c (gtk_plug_focus): Send the focus on
+Mon Aug 25 23:21:43 2003 Kristian Rietveld <kris@gtk.org>
+
+ Merge from stable.
+
+ Fixes #115871, reported by Michael Natterer.
+
+ * gtk/gtktreeprivate.h: added GtkTreeSelectMode enum,
+ added ctrl_pressed and shift_pressed bitfields,
+ (_gtk_tree_selection_internal_select_node): replace GdkModifierType
+ arg with GtkTreeSelectMode.
+
+ * gtk/gtktreeselection.c (gtk_tree_selection_set_mode),
+ (gtk_tree_selection_select_path), (gtk_tree_selection_unselect_path),
+ (_gtk_tree_selection_internal_select_node): all updated for
+ GdkModifierType -> GtkTreeSelectMode move.
+
+ * gtk/gtktreeview.c (gtk_tree_view_button_press): set ctrl_pressed
+ and shift_pressed around selection handling block,
+ (gtk_tree_view_real_select_cursor_row),
+ (gtk_tree_view_real_toggle_cursor_row),
+ (gtk_tree_view_real_selection_cursor_parent),
+ (gtk_tree_view_real_set_cursor): use ctrl_pressed and shift_pressed,
+ instead of checking the event state. And also updates for the
+ GdkModifierType -> GtkTreeSelectMode move.
+
Wed Aug 20 21:26:49 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkplug.c (gtk_plug_focus): Send the focus on
+Mon Aug 25 23:21:43 2003 Kristian Rietveld <kris@gtk.org>
+
+ Merge from stable.
+
+ Fixes #115871, reported by Michael Natterer.
+
+ * gtk/gtktreeprivate.h: added GtkTreeSelectMode enum,
+ added ctrl_pressed and shift_pressed bitfields,
+ (_gtk_tree_selection_internal_select_node): replace GdkModifierType
+ arg with GtkTreeSelectMode.
+
+ * gtk/gtktreeselection.c (gtk_tree_selection_set_mode),
+ (gtk_tree_selection_select_path), (gtk_tree_selection_unselect_path),
+ (_gtk_tree_selection_internal_select_node): all updated for
+ GdkModifierType -> GtkTreeSelectMode move.
+
+ * gtk/gtktreeview.c (gtk_tree_view_button_press): set ctrl_pressed
+ and shift_pressed around selection handling block,
+ (gtk_tree_view_real_select_cursor_row),
+ (gtk_tree_view_real_toggle_cursor_row),
+ (gtk_tree_view_real_selection_cursor_parent),
+ (gtk_tree_view_real_set_cursor): use ctrl_pressed and shift_pressed,
+ instead of checking the event state. And also updates for the
+ GdkModifierType -> GtkTreeSelectMode move.
+
Wed Aug 20 21:26:49 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkplug.c (gtk_plug_focus): Send the focus on
+Mon Aug 25 23:21:43 2003 Kristian Rietveld <kris@gtk.org>
+
+ Merge from stable.
+
+ Fixes #115871, reported by Michael Natterer.
+
+ * gtk/gtktreeprivate.h: added GtkTreeSelectMode enum,
+ added ctrl_pressed and shift_pressed bitfields,
+ (_gtk_tree_selection_internal_select_node): replace GdkModifierType
+ arg with GtkTreeSelectMode.
+
+ * gtk/gtktreeselection.c (gtk_tree_selection_set_mode),
+ (gtk_tree_selection_select_path), (gtk_tree_selection_unselect_path),
+ (_gtk_tree_selection_internal_select_node): all updated for
+ GdkModifierType -> GtkTreeSelectMode move.
+
+ * gtk/gtktreeview.c (gtk_tree_view_button_press): set ctrl_pressed
+ and shift_pressed around selection handling block,
+ (gtk_tree_view_real_select_cursor_row),
+ (gtk_tree_view_real_toggle_cursor_row),
+ (gtk_tree_view_real_selection_cursor_parent),
+ (gtk_tree_view_real_set_cursor): use ctrl_pressed and shift_pressed,
+ instead of checking the event state. And also updates for the
+ GdkModifierType -> GtkTreeSelectMode move.
+
Wed Aug 20 21:26:49 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkplug.c (gtk_plug_focus): Send the focus on
GTK_TREE_VIEW_IN_COLUMN_DRAG = 1 << 7
} GtkTreeViewFlags;
+typedef enum
+{
+ GTK_TREE_SELECT_MODE_TOGGLE = 1 << 0,
+ GTK_TREE_SELECT_MODE_EXTEND = 1 << 1
+}
+GtkTreeSelectMode;
+
enum
{
DRAG_COLUMN_WINDOW_STATE_UNSET = 0,
/* hint to display rows in alternating colors */
guint has_rules : 1;
guint mark_rows_col_dirty : 1;
+
+ guint ctrl_pressed : 1;
+ guint shift_pressed : 1;
/* interactive search */
guint enable_search : 1;
GtkRBNode *node,
GtkRBTree *tree,
GtkTreePath *path,
- GdkModifierType state,
+ GtkTreeSelectMode mode,
gboolean override_browse_mode);
gboolean _gtk_tree_view_find_node (GtkTreeView *tree_view,
GtkTreePath *path,
node,
tree,
anchor_path,
- 0,
+ 0,
FALSE);
if (anchor_path)
gtk_tree_path_free (anchor_path);
{
GtkRBNode *node;
GtkRBTree *tree;
- GdkModifierType state = 0;
gboolean ret;
+ GtkTreeSelectMode mode = 0;
g_return_if_fail (GTK_IS_TREE_SELECTION (selection));
g_return_if_fail (selection->tree_view != NULL);
return;
if (selection->type == GTK_SELECTION_MULTIPLE)
- state = GDK_CONTROL_MASK;
+ mode = GTK_TREE_SELECT_MODE_TOGGLE;
_gtk_tree_selection_internal_select_node (selection,
node,
tree,
path,
- state,
+ mode,
FALSE);
}
node,
tree,
path,
- GDK_CONTROL_MASK,
+ GTK_TREE_SELECT_MODE_TOGGLE,
TRUE);
}
GtkRBNode *node,
GtkRBTree *tree,
GtkTreePath *path,
- GdkModifierType state,
+ GtkTreeSelectMode mode,
gboolean override_browse_mode)
{
gint flags;
else if (selection->type == GTK_SELECTION_SINGLE &&
anchor_path && gtk_tree_path_compare (path, anchor_path) == 0)
{
- if ((state & GDK_CONTROL_MASK) == GDK_CONTROL_MASK)
+ if ((mode & GTK_TREE_SELECT_MODE_TOGGLE) == GTK_TREE_SELECT_MODE_TOGGLE)
{
dirty = gtk_tree_selection_real_unselect_all (selection);
}
}
else if (selection->type == GTK_SELECTION_MULTIPLE)
{
- if (((state & GDK_SHIFT_MASK) == GDK_SHIFT_MASK) && (anchor_path == NULL))
+ if ((mode & GTK_TREE_SELECT_MODE_EXTEND) == GTK_TREE_SELECT_MODE_EXTEND
+ && (anchor_path == NULL))
{
if (selection->tree_view->priv->anchor)
gtk_tree_row_reference_free (selection->tree_view->priv->anchor);
gtk_tree_row_reference_new_proxy (G_OBJECT (selection->tree_view), selection->tree_view->priv->model, path);
dirty = gtk_tree_selection_real_select_node (selection, tree, node, TRUE);
}
- else if ((state & (GDK_CONTROL_MASK|GDK_SHIFT_MASK)) == (GDK_SHIFT_MASK|GDK_CONTROL_MASK))
+ else if ((mode & (GTK_TREE_SELECT_MODE_EXTEND | GTK_TREE_SELECT_MODE_TOGGLE)) == (GTK_TREE_SELECT_MODE_EXTEND | GTK_TREE_SELECT_MODE_TOGGLE))
{
gtk_tree_selection_select_range (selection,
anchor_path,
path);
}
- else if ((state & GDK_CONTROL_MASK) == GDK_CONTROL_MASK)
+ else if ((mode & GTK_TREE_SELECT_MODE_TOGGLE) == GTK_TREE_SELECT_MODE_TOGGLE)
{
flags = node->flags;
if (selection->tree_view->priv->anchor)
else
dirty |= gtk_tree_selection_real_select_node (selection, tree, node, TRUE);
}
- else if ((state & GDK_SHIFT_MASK) == GDK_SHIFT_MASK)
+ else if ((mode & GTK_TREE_SELECT_MODE_EXTEND) == GTK_TREE_SELECT_MODE_EXTEND)
{
dirty = gtk_tree_selection_real_unselect_all (selection);
dirty |= gtk_tree_selection_real_modify_range (selection,
*/
if (event->type == GDK_BUTTON_PRESS && event->button == 1)
{
+ if ((event->state & GDK_CONTROL_MASK) == GDK_CONTROL_MASK)
+ tree_view->priv->ctrl_pressed = TRUE;
+ if ((event->state & GDK_SHIFT_MASK) == GDK_SHIFT_MASK)
+ tree_view->priv->shift_pressed = TRUE;
+
focus_cell = _gtk_tree_view_column_get_cell_at_pos (column, event->x - background_area.x);
if (focus_cell)
gtk_tree_view_column_focus_cell (column, focus_cell);
{
gtk_tree_view_real_set_cursor (tree_view, path, TRUE, TRUE);
}
+
+ tree_view->priv->ctrl_pressed = FALSE;
+ tree_view->priv->shift_pressed = FALSE;
}
/* the treeview may have been scrolled because of _set_cursor,
GtkRBTree *cursor_tree = NULL;
GtkRBNode *cursor_node = NULL;
GtkTreePath *cursor_path = NULL;
- GdkModifierType state = 0;
- cursor_path = NULL;
+ GtkTreeSelectMode mode = 0;
if (! GTK_WIDGET_HAS_FOCUS (tree_view))
return FALSE;
return FALSE;
}
- gtk_get_current_event_state (&state);
-
- if (! (state & GDK_SHIFT_MASK) &&
- start_editing &&
+ if (!tree_view->priv->shift_pressed && start_editing &&
tree_view->priv->focus_column)
{
if (gtk_tree_view_start_editing (tree_view, cursor_path))
return TRUE;
}
}
+
+ if (tree_view->priv->ctrl_pressed)
+ mode |= GTK_TREE_SELECT_MODE_TOGGLE;
+ if (tree_view->priv->shift_pressed)
+ mode |= GTK_TREE_SELECT_MODE_EXTEND;
+
_gtk_tree_selection_internal_select_node (tree_view->priv->selection,
cursor_node,
cursor_tree,
cursor_path,
- state,
+ mode,
FALSE);
gtk_tree_view_clamp_node_visible (tree_view, cursor_tree, cursor_node);
gtk_widget_grab_focus (GTK_WIDGET (tree_view));
_gtk_tree_view_queue_draw_node (tree_view, cursor_tree, cursor_node, NULL);
- if (! (state & GDK_SHIFT_MASK))
- gtk_tree_view_row_activated (tree_view, cursor_path, tree_view->priv->focus_column);
+ if (!tree_view->priv->shift_pressed)
+ gtk_tree_view_row_activated (tree_view, cursor_path,
+ tree_view->priv->focus_column);
gtk_tree_path_free (cursor_path);
cursor_node,
cursor_tree,
cursor_path,
- GDK_CONTROL_MASK,
+ GTK_TREE_SELECT_MODE_TOGGLE,
FALSE);
gtk_tree_view_clamp_node_visible (tree_view, cursor_tree, cursor_node);
cursor_node,
cursor_tree,
cursor_path,
- 0,
+ 0,
FALSE);
}
{
GtkRBTree *tree = NULL;
GtkRBNode *node = NULL;
- GdkModifierType state = 0;
if (gtk_tree_row_reference_valid (tree_view->priv->cursor))
{
}
gtk_tree_row_reference_free (tree_view->priv->cursor);
- gtk_get_current_event_state (&state);
tree_view->priv->cursor = gtk_tree_row_reference_new_proxy (G_OBJECT (tree_view),
tree_view->priv->model,
GtkRBTree *new_tree = NULL;
GtkRBNode *new_node = NULL;
- if (clear_and_select && !((state & GDK_CONTROL_MASK) == GDK_CONTROL_MASK))
- _gtk_tree_selection_internal_select_node (tree_view->priv->selection,
- node, tree, path,
- state, FALSE);
+ if (clear_and_select && !tree_view->priv->ctrl_pressed)
+ {
+ GtkTreeSelectMode mode = 0;
+
+ if (tree_view->priv->ctrl_pressed)
+ mode |= GTK_TREE_SELECT_MODE_TOGGLE;
+ if (tree_view->priv->shift_pressed)
+ mode |= GTK_TREE_SELECT_MODE_EXTEND;
+
+ _gtk_tree_selection_internal_select_node (tree_view->priv->selection,
+ node, tree, path, mode,
+ FALSE);
+ }
/* We have to re-find tree and node here again, somebody might have
* cleared the node or the whole tree in the GtkTreeSelection::changed